Skip to content

Comments

Add internal APIs for the testing purpose#169

Merged
s-vitaliy merged 3 commits intomainfrom
add-mock-stream-definition-for-tests
Jan 6, 2026
Merged

Add internal APIs for the testing purpose#169
s-vitaliy merged 3 commits intomainfrom
add-mock-stream-definition-for-tests

Conversation

@s-vitaliy
Copy link
Contributor

@s-vitaliy s-vitaliy commented Jan 6, 2026 •
edited
Loading

Part of #144

Scope

This PR adds the generated stream definition which can be used for testing.

Checklist

  • GitHub issue exists for this change.
  • Unit tests added and they pass.
  • Line Coverage is at least 80%.
  • Review requested on latest commit.

@s-vitaliy s-vitaliy requested a review from a team as a code owner January 6, 2026 09:51
@github-actions
Copy link

github-actions bot commented Jan 6, 2026 •
edited
Loading

go-test-coverage report:

  below threshold:				coverage:	threshold:
  configuration/configuration.go		 0.0% (0/18)	70%
  telemetry/app_metrics.go			 0.0% (0/9)	70%

Package coverage threshold (70%) satisfied:	FAIL
  below threshold:				coverage:	threshold:
  telemetry					 0.0% (0/9)	70%
  configuration					 0.0% (0/18)	70%

Total coverage threshold (75%) satisfied:	FAIL
Total test coverage: 60.0% (54/90)

Files with uncovered lines:
  file:							coverage:	uncovered lines:
  configuration/configuration.go			0%		17-20 22-24 27-36 39-41 43-47 52-55 60
  services/controllers/stream_class/stream_class.go	85.7%		40-42 47-49 67-70 90-93 103-106 141-144
  telemetry/app_metrics.go				0%		18-22 24 53-56 60-63 65

@s-vitaliy s-vitaliy merged commit 1d4fba8 into main Jan 6, 2026
1 of 2 checks passed
@s-vitaliy s-vitaliy deleted the add-mock-stream-definition-for-tests branch January 6, 2026 10:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@george-zubrienko george-zubrienko george-zubrienko approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@s-vitaliy @george-zubrienko